-
Notifications
You must be signed in to change notification settings - Fork 25k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated tutorial text #117940
Updated tutorial text #117940
Conversation
💚 CLA has been signed |
Documentation preview: |
I signed the document now what |
Shalll I raise a PR for image change as well? |
Pinging @elastic/es-data-management (Team:Data Management) |
Pinging @elastic/es-docs (Team:Docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this @satyyam11, but we need to update the images as part of this PR in addition to changing the text. Could you make those image updates as well?
Elastic Stack deployments on {ess} include a hot tier by default. To add a warm | ||
tier, edit your deployment and click **Add capacity** for the warm data tier. | ||
+ | ||
Elastic Stack deployments on {ess} automatically include hot and warm tiers. If needed, you can add additional warm capacity by editing your deployment and selecting **Add capacity** for the warm data tier. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They do not automatically include a warm
tier. They include a hot
and content
tier.
Elastic Stack deployments on {ess} automatically include hot and warm tiers. If needed, you can add additional warm capacity by editing your deployment and selecting **Add capacity** for the warm data tier. | |
Elastic Stack deployments on {ess} automatically include hot and content tiers. If needed, you can add additional warm capacity by editing your deployment and selecting **Add capacity** for the warm data tier. |
[role="screenshot"] | ||
image::images/ilm/tutorial-ilm-ess-add-warm-data-tier.png[Add a warm data tier to your deployment] | ||
image::images/ilm/tutorial-ilm-add-warm-data-tier.png[Add a warm data tier to your deployment] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason you changed this image link? As part of this the pngs should be updated to show the newer UI.
phase. Then disable **Use recommended defaults** to display the rollover | ||
settings. | ||
phase. Adjust the **Minimum primary shard size** and other rollover settings as needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A user will still need to uncheck the "Use recommended defaults" right?
|
||
[role="screenshot"] | ||
image::images/ilm/tutorial-ilm-hotphaserollover-default.png[View rollover defaults] | ||
image::images/ilm/tutorial-ilm-hotphaserollover-new.png[View rollover defaults] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image doesn't exist, do you need to add it to git so it's committed for this PR?
+ | ||
-- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a lot of unnecessary whitespace changes in this file, can you undo those?
Any updates? |
This should be closed because it's been superseded by #118148 |
This PR updates the Elasticsearch ILM tutorial to make things clearer and easier to follow.
I’ve cleaned up the steps for customizing ILM policies, explained the rollover, warm, and delete phases a bit more, and made sure everything’s easy to understand.
Basically, it's to help people better manage their ILM policies based on their needs, without all the confusion.